Skip to content

[Spec 650] Team page: show issue/PR details and activity feed#651

Merged
waleedkadous merged 6 commits intomainfrom
builder/aspir-650-team-page-show-issue-pr-detail
Apr 1, 2026
Merged

[Spec 650] Team page: show issue/PR details and activity feed#651
waleedkadous merged 6 commits intomainfrom
builder/aspir-650-team-page-show-issue-pr-detail

Conversation

@waleedkadous
Copy link
Copy Markdown
Contributor

Summary

  • Expand team page member cards to show individual issue/PR titles with clickable GitHub links (replacing count-only display)
  • Add combined activity feed showing recent merged PRs and closed issues across all members
  • Minimal backend fix: add url to GraphQL query for recent activity items

Changes

  • team-github.ts: Add url to merged PR and closed issue GraphQL fragments + types + parsing
  • api.ts: Update frontend TeamMemberGitHubData type with url on activity items
  • TeamView.tsx: Expand MemberCard with issue/PR title lists; add ActivityFeed component with relativeDate helper
  • index.css: Styles for item lists, truncation, activity feed layout
  • team-github.test.ts: Add explicit url assertions for recent activity
  • activityFeed.test.ts: 8 new tests for relativeDate and buildActivityFeed

Testing

  • All unit tests passing (80 backend + 8 dashboard)
  • Build passes
  • TypeScript clean

Spec

Link: codev/specs/650-team-page-show-issue-pr-detail.md

Review

Link: codev/reviews/650-team-page-show-issue-pr-detail.md

@waleedkadous
Copy link
Copy Markdown
Contributor Author

Architect Review

Nice work — clean implementation that makes good use of data we were already fetching.

What changed:

  • Member cards now show clickable issue/PR titles instead of just counts
  • New activity feed section aggregates merged PRs and closed issues across all members
  • Minimal backend fix adds url to GraphQL fragments for recent activity (correct catch from consultation)
  • Pure functions (relativeDate, buildActivityFeed) cleanly separated and tested

Looks good. Go ahead and merge.

@waleedkadous waleedkadous merged commit c86fa1a into main Apr 1, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant